-
Notifications
You must be signed in to change notification settings - Fork 695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add litep2p network protocol benches #6455
Conversation
As a side note to the PR description: how to interpret the plot axes? Naive understanding brings the question up why the time is the same for 10^3 bytes and for 10^6 bytes? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Amazing work here Andrei! 🙏
Either the worker initialization or the size difference is not significant. |
/cmd prdoc --audience node_dev --bump silent |
Description
Add support to run networking protocol benchmarks with litep2p backend.
Now we can compare the work of both libp2p and litep2p backends for notifications and request-response protocols.
Next step: extract worker initialization from the benchmark loop.
Example run on local machine
Integration
Does not affect downstream projects.
Review Notes
polkadot-sdk/substrate/client/network/src/litep2p/service.rs
Lines 510 to 520 in d4d9502
This method should be implemented to run request benchmarks.